Skip to content

Support for adding scoped CSS without triggering global refresh #1999

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 14, 2019

Conversation

joshcomley
Copy link
Contributor

As per the problem I reported here:
NativeScript/theme#179

When using component scoped CSS there is no need to trigger a full global refresh of the styles.

This change does not affect performance or function of using encapsulation: ViewEncapsulation.None on a component.

This PR goes hand in hand with a modification to tns-core-modules to support the below changes.

PR Checklist

What is the current behavior?

What is the new behavior?

Fixes/Implements/Closes #[Issue Number].

@joshcomley
Copy link
Contributor Author

I am happy to resubmit this PR following the above guidelines (I am not entirely sure how to compose the PR title, for example), but would somebody be able to review the code to ensure it's a valid PR otherwise? Many thanks

Fixed profiling naming inconsistencies

Co-Authored-By: Martin Yankov <[email protected]>
Naming consistency fix

Co-Authored-By: Martin Yankov <[email protected]>
@MartoYankov
Copy link
Contributor

@VladimirAmiorkov This PR depends on NativeScript/NativeScript#7907 , which will be released with {N} 6.2.

@VladimirAmiorkov
Copy link
Contributor

test tns_core_modules#next

@VladimirAmiorkov
Copy link
Contributor

VladimirAmiorkov commented Oct 9, 2019

All checked have passed. This PR is ready for merge once [email protected] is about to be fully released.

@joshcomley
Copy link
Contributor Author

joshcomley commented Oct 9, 2019

@VladimirAmiorkov thanks - does that mean this will go live at roughly the same time as 6.2.0?

(edit: I had written the completely wrong question)

@VladimirAmiorkov
Copy link
Contributor

@joshcomley Yes this will make it in time for NativeScript 6.2.0.

@joshcomley
Copy link
Contributor Author

@VladimirAmiorkov is there an estimated date for the 6.2.0 release? I understand this isn't exactly the thread to ask, but I'm not sure where is!

@MartoYankov
Copy link
Contributor

@joshcomley We are aiming for the end of the month.

@elena-p
Copy link
Contributor

elena-p commented Oct 14, 2019

test tns_core_modules#next

@elena-p elena-p merged commit 662c122 into NativeScript:master Oct 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants